Skip to content

Say in CLAUDE.md that the shared math basics live in mcs - #671

Merged
dem-extra1 merged 2 commits into
mainfrom
claude/mcs60-phase7-claude-md
Nov 6, 2026
Merged

dem-extra1 merged 2 commits into
mainfrom
claude/mcs60-phase7-claude-md

Conversation

@dem-extra1

Copy link
Copy Markdown
Contributor

Part of Morrison-Lab/mcs#60 (phase 7).

The shared math basics moved to mcs and the matching mds pages are redirect stubs (#661).
CLAUDE.md still described mds as holding that material, so an agent could add to the stubs.
This adds a short paragraph saying the content lives in mcs, the first site in the course sequence, and where to link and add new content.


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FWgYrGcBYiCDELfrR89zk4
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FWgYrGcBYiCDELfrR89zk4
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Claude finished review — View run


I fetched the PR live just now. #671 is open, not a draft and not merged. Its head is still 2c0e67fb4e0655d727274d68969caa57c146370b, the commit I reviewed, so the review stands.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "2c0e67fb4e0655d727274d68969caa57c146370b",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The only changed path is CLAUDE.md, where a paragraph was added; sets.qmd confirmed as a redirect stub to the mcs site, and no defects were found in the added text.",
  "holistic_assessment": "The guidance-only change is well scoped and consistent with the repository's URL-interface rule; I did not check every listed topic for a stub or the mcs#60 reference, so those remain unverified."
}

Reviewed commit: 2c0e67f

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

💰 Cost: $0.2048 (review) — run

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.2
🚀 Preview available at: https://Morrison-Lab.github.io/mds/pr-preview/pr-671/

Built to branch gh-pages at 2026-11-06 05:37 UTC.

@dem-extra1
dem-extra1 merged commit 1ee8354 into main Nov 6, 2026
42 checks passed

Copy link
Copy Markdown
Contributor Author

Merged on Ezra's mwc (2026-11-05 23:46 PST) with the checker reporting FULLY CLEAN on head 2c0e67f.
The cross-harness review was waived: the OpenCode Go endpoint returned its monthly-limit error, and the codex, agy, gemini, cursor-agent and opencode CLIs are not installed in this session.
This follows Ezra's 2026-10-16 rule that a clean Claude CI review plus green CI suffices while the cross-harness reviewers are out.


Posted by Claude Code (AI agent) --- not written by a human.


Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants